Skip to content

refactor: adapters are in fact services + rename them#438

Merged
xavier-brochard merged 7 commits into
mainfrom
move-subscriptions-to-service-layer
Jul 1, 2025
Merged

refactor: adapters are in fact services + rename them#438
xavier-brochard merged 7 commits into
mainfrom
move-subscriptions-to-service-layer

Conversation

@xavier-brochard

@xavier-brochard xavier-brochard commented Jul 1, 2025

Copy link
Copy Markdown
Contributor
  • Refactor and rename classes that manage websocket connections and subscriptions, as @darioAnongba mentioned it didn't belong into src/infrastructure/.
  • Rename NetworkWithRpcUrls > NetworkConfig as @zone-live suggested here

@xavier-brochard
xavier-brochard marked this pull request as ready for review July 1, 2025 10:58
cursor[bot]

This comment was marked as outdated.

@sonarqubecloud

sonarqubecloud Bot commented Jul 1, 2025

Copy link
Copy Markdown

@xavier-brochard
xavier-brochard merged commit 3de1292 into main Jul 1, 2025
10 checks passed
@xavier-brochard
xavier-brochard deleted the move-subscriptions-to-service-layer branch July 1, 2025 12:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants